Skip to content

[ready to merge] docs(chrome-extension): salvage useful bits from #588#643

Merged
trunk-staging-io[bot] merged 1 commit intomainfrom
sam-gutentag/chrome-extension-polish
May 7, 2026
Merged

[ready to merge] docs(chrome-extension): salvage useful bits from #588#643
trunk-staging-io[bot] merged 1 commit intomainfrom
sam-gutentag/chrome-extension-polish

Conversation

@samgutentag
Copy link
Copy Markdown
Member

Summary

Pulls forward the small bits worth keeping from #588 (Eli's original Chrome Extension docs PR), since the page itself has evolved well past that draft via several subsequent merged PRs (#593, #595, #598, #626, #639).

Changes

  • Add a frontmatter description: so GitBook renders proper page metadata for SEO and navigation previews
  • Add an inline pointer to the Submit and cancel pull requests reference page after the Submit steps, so readers can drill into the full lifecycle from this entry point
  • Replace the FAQ link to a hard-coded GitHub commit URL with a relative path to getting-started/, matching the rest of the docs

Test plan

  • Verify the page metadata description renders in GitBook preview
  • Confirm the new inline link to using-the-queue/reference.md resolves correctly
  • Confirm the FAQ link to getting-started/ resolves correctly

Related

Pulls forward the small bits worth keeping from Eli's original Chrome
Extension docs PR, since the page itself has evolved well past that
draft via several subsequent merged PRs.

- Add a frontmatter description so GitBook renders proper metadata
- Add an inline pointer to the Submit and cancel pull requests
  reference page after the Submit steps, so readers can drill into
  the full lifecycle from this entry point
- Replace the FAQ link to a hard-coded GitHub commit URL with a
  relative path to getting-started, matching the rest of the docs

Closes the conflict-stale #588 in favor of these targeted edits on
top of the current main.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@trunk-staging-io
Copy link
Copy Markdown

trunk-staging-io Bot commented May 7, 2026

😎 Merged successfully - details.

@claude
Copy link
Copy Markdown
Contributor

claude Bot commented May 7, 2026

Claude finished @samgutentag's task in 1m 17s —— View job


Review: PR #643 — salvage useful bits from #588

  • Read changed file and PR diff
  • Verify relative link targets resolve
  • Check for typos, grammar, formatting
  • Check description frontmatter style consistency
  • Post review feedback

Summary

Clean, low-risk salvage PR. All three changes do what the description claims, and I don't see any blocking issues.

What's good

  • Frontmatter description: (line 2) — single-line form is consistent with other pages in this directory (e.g., webhooks.md). Content reads well and is SEO-appropriate at ~130 chars.
  • New inline pointer (line 36) — relative path using-the-queue/reference.md resolves correctly, and the link text "Submit and cancel pull requests" exactly matches that page's H1.
  • FAQ link fix (line 75) — getting-started/ resolves to merge-queue/getting-started/README.md ("Getting Started"), and switching from a hard-coded GitHub commit URL to a relative path is the right call — it matches every other internal link in this file and won't break when the source moves.

Minor nit (non-blocking, posted inline on line 36)

The first clause of the new sentence ("...same backend as the /trunk merge comment...so behavior is identical") slightly duplicates the existing FAQ entry below it. Consider trimming to just the "See [Submit and cancel pull requests]…" pointer. Optional.

Verdict

LGTM. Approving the spirit; flagging the inline nit for the author's call.
Branch

3. Optionally choose a [priority](optimizations/priority-merging.md) before submitting.
4. If batching is enabled for the repository, you can toggle **Skip batching** to enqueue this PR without grouping it into a batch — useful for hotfixes or PRs that need to merge without waiting for a batch window.

Submission goes through the same backend as the `/trunk merge` comment and the Trunk web app, so behavior is identical. See [Submit and cancel pull requests](using-the-queue/reference.md) for the full lifecycle.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit, non-blocking: the first half of this sentence ("Submission goes through the same backend as the /trunk merge comment and the Trunk web app, so behavior is identical") overlaps closely with the FAQ entry below ("How does the extension differ from the /trunk merge comment?"), which says "Both go through the same Trunk Merge Queue backend." If you want to deduplicate, you could trim this to just the pointer — e.g., "See Submit and cancel pull requests for the full lifecycle." — and let the FAQ keep the equivalence claim. Up to you; either way is fine.

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Auto-approved: Claude code review passed.

@samgutentag
Copy link
Copy Markdown
Member Author

Verification status (2026-05-07): live

Verified: customers can use this. Ready to publish.

Ready to merge as soon as review is complete.

@samgutentag samgutentag changed the title docs(chrome-extension): salvage useful bits from #588 [ready to merge] docs(chrome-extension): salvage useful bits from #588 May 7, 2026
@trunk-staging-io trunk-staging-io Bot merged commit 7a1a54d into main May 7, 2026
7 checks passed
@trunk-staging-io trunk-staging-io Bot deleted the sam-gutentag/chrome-extension-polish branch May 7, 2026 22:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant